Code smells" 




Refaktoryzacja polega na poprawianiu organizacj 
i polepszaniu czytelności kodu źródłowego. 



Ogólnie chodzi o poprawienie cech jakości kodu 
źródłowego i likwidowanie „code smells"- 
niezręcznych konstrukcji programistycznych. 



Za i przeciw refaktoryzacji, 




° stosujemy kiedy: 

° 1. Zmieniliśmy coś w kodzie (three strikes and refactor). 

° 2. Była zmiana w funkcjonalności. 

° 3. Podczas naprawiania błędu (fixing a bug). 

° 4. Podczas inspekcji kodu. 

' Nie stosujemy, gdy: 

° 1. Mamy mało czasu (at close deadlines). 
1.2. Niestabilny kod. 



Rodzaje 'code smells 



1. Powtórzenie kodu. 

2. Zbyt wiele komentarzy. 
S.Type codę". 

4. Obszerne klasy. 

5. Długie listy parametrów. 

6. Metody zawierające zbyt wiele kodu. 

7. Niektóre zmienne nie zawsze są stosowane. 



Przykład 1 



class Shape { 

D finał static int TYPELINE = 0; 
D finał static int TYPERECTANGLE = 1; 
D finał static int TYPECIRCLE = 2; 
D int shapeType; 

D //starting point of the line. 

D //lower left corner of the rectangle. 

D //center of the circle. 

D Point pl; 

O //ending point of the line. 

D //upper right corner of the rectangle. 

O //not used for the circle. 

D Point p2; 

D int radius; 

} 



class CADApp { 

void drawShapes (Graphics graphics, Shape shapes[]) { 
D for (int i = 0; i < shapes . length; i++) { 
° switch ( shapes [ i ] . getType ( ) ) { 
° case Shape . TYPELINE : 

° graphics . drawLine (shapes [i] . getPl ( ) , shapes [ i ]. getP2 ()) ; 
° break; 
" case Shape . TYPERECTANGLE : 
° //draw the four edges . 
° graphics . drawLine (...); 
° graphics . drawLine (...); 
° graphics . drawLine (...); 
° graphics . drawLine (...); 
° break; 
° case Shape . TYPECIRCLE : 

° graphics . drawCircle (shapes [i] . getPl O , shapes [ i ]. getRadius () ) 
° break; 
° } 
" } 
D } 

D } 



Problem 1 : Kod będzie w przyszłości zmieniany 



class Shape { 

D finał static int TYPELINE = 0; 

Q finał static int TYPERECTANGLE = 1; 

Q finał static int TYPECIRCLE = 2; 

Q finał static int TYPETRIANGLE = 3; 

D int shapeType; 

D //starting point of the łine. 

D //łower łeft corner of the rectangłe. 

Q //center of the circłe. 

D Point pl; 

Q //ending point of the łine. 

D //upper right corner of the rectangłe. 

D //not used for the circłe. 

D Point p2; 

D //third point of the triangle . 

D Point p3; 

D int radius; 
} 



class CADApp { I 

void drawShapes (Graphics graphics, Shape shapes[]) { 

D for (int i = 0; i < shapes . length; i++) { 
° switch ( shapes [ i ] . getType ( ) ) { 
" case Shape . TYPELINE : 

" graphics . drawLine (shapes [i] . getPl ( ) , shapes [ i ]. getP2 ()) ; 
" break; 
" case Shape . TYPERECTANGLE : 

" //draw the four edges. 



" graphics . drawLine (...); 
" break; 

case Shape . TYPECIRCLE : 

" graphics . drawCircle ( shapes [ i ] . getPl ( ) , shapes [ i ] . getRadius ( ) ) ; 
" break; 

case Shape . TYPETRIANGLE : 

" graphics . drawLine (shapes [i] . getPl O , shapes [i] . getP2 ()) ; 
" graphics . drawLine (shapes [i] . getP2 O , shapes [i] . getP3 ()) ; 
" graphics . drawLine (shapes [i] . getP3 O , shapes [i] . getPl ()) ; 
" break; 



graphics . drawLine ( . . . 
graphics . drawLine ( . . . 
graphics . drawLine ( . . . 



Dodanie możliwości 
rysowania nowej figury(trójkąt) 
wymusiło modyfikacje obu klas. 



} 



} 



a } 



° } 



Smell codę" nr 1 




cjlass Shape { 
finał static int TYPELINE = 0; 
finał static int TYPERECTANGLE =1; 
finał static int TYPECłRCLE = 2; 
int shapeType; 
}| 



Kod takiego typu, 
powinien być poważnym 
ostrzeżeniem ,że w 
przyszłości mogą pojawić 
się problemy. 



Przekształcenie 'type code' na klasy 



° Problem: 

° Klasa posiada pole o skończonej liczbie wartości, którego 

wartość nie 
° wpływa na zachowanie 
° Cel 

° Przekształcenie pola w nową klasę 

1) Mechanika 

2) Utwórz nową klasę 

3) Dodaj do klasy źródłowej pole typu tej klasy i zainicjuj je 

4) dla każdej metody w klasie źródłowej, która korzysta z 
oryginalnego 

l)pola stanu, utwórz jej odpowiednik korzystający z nowego 
pola 

5) zmień metody ,tak aby korzystały z nowych metod 

fiy^knmnilui i nr7RtR.9tiJi 



class Shape { 
} 

class Line extends Shape { 

D 

• • • 

} 

class Rectangle extends Shape { 

D 

• • • 

} 

class Circle extends Shape { 

D 

• • • 

} 



Smell code' nr 2 



class Shape { 

D 

• • • 

n Point pl; 
o Point p2 ; 
o int radius; 

} 



Pojawia się zmienna (int radius), która 
nie zawsze jest używana. 



Smell code' nr 3 




class Shape { 

□ 

• • • 

D Point pl; 
Q Point p2 ; 
Q int radius; 

} 



Zmienne pl,p2 powinny mieć inne nazwy 
ze względu na przejrzystość kodu. 



Usunięcie 'smell code' nrl-3 



class Shape { 
} 

class Line extends Shape { 
Q Point startPoint; 
Q Point endPoint; 

} 

class Rectangle extends Shc 
Q Point lowerLeftCorner ; 
D Point upperRightCorner ; 

} 

class Circle extends Shape { 
D Point center; 
Q int radius; 



Usunięcie 'smell code' nr 1 za 
pomocą stworzenie podklas, klasy Shape. 

Zmienna int radius została dodana do 
Podklasy Circle dzięki temu zostanie 
Użyta tylko w potrzebie nie zajmując 
zbędnie pamięci. 

Nazwy zmiennych pl,p2 zostały zmienione 
na bardziej przejrzyste.? 



Poprawiony kod 



interface Shape { 

Q void dr aw (Graphics graphics) ; 

} 

class Line implements Shape { 
Q Point startPoint; 
Q Point endPoint; 

Q void draw (Graphics graphics) { 
° graphics . drawLine (getStartPoint ( ) 
getEndPoint () ) ; 

' } 
} 

class Circle implements Shape { 
Q Point center; 
D int radius; 



class Rectangle implements Shape { 

° Point lowerLeftCorner; 

° Point upperRightCorner; 
Q void dr aw (Graphics graphics) { 

° graphics . drawLine (...); 

° graphics . drawLine (...); 

° graphics . drawLine (...); 

° graphics . drawLine (...); 

° } 

} 

class CADApp { 

void drawShapes (Graphics graphics, 
Q for (int i = 0; i < shapes . length 
o shapes [i] . draw (graphics) ; 

° } 
a } 
} 



Klasa^ADAp pozostała 
uproszczona.do minimunn, dzięki 
wprowadzeniu interfejsu. 

Kod jest przejrzysty.dzięki temu ,że jest 
więcej mniejszych klas, które są 
zrozumiałe, a w wypadku jakiś 
problemów poprawa ich nie powinna 

stanowić proMerriu>. 

1 ^ ^ 



Shape shapes [ ] ) { 
; i++) { 



Zadanie: 




class SurveyData { I 
String path; //save the data to this file. 
boolean hidden; //should the file be hidden? 

//set the path to save the data according to the type of data (t) 
Q void set SavePath ( int t) { 
D if (t==0) { //raw data. 

° path = " c : /applicat ion/data/raw . dat " ; 

° hidden = true; 
o } else if (t==l) { //cleaned up data. 

° Path = " c : /applicat ion/data/cleanedUp . dat " ; 

° hidden = true; 
o } else if (t==2) { //processed data. 

° Path = "c : /application/data/processed. dat " ; 

° hidden = true; 
o } else if (t==3) { //data ready for publication. 

° Path = " c : /applicat ion/data/publicat ion . dat " ; 

° hidden = false; 

a } 
° } 

} 



Przykładowe rozwiązanie 



class SurveyDataType { 
String baseFileName; 
boolean hideDataFile; 

SurveyDataType (String baseFileName, boolean hideDataFile) { 

this . baseFileName = baseFileName; 
this . hideDataFile = hideDataFile; 
} 

String getSavePath () { 

return " c : /applicat ion/dat a/ " + baseFileName + „.dat"; 
} 

^ static SurveyDataType rawDataType = new SurveyDataType ("raw" , true) ; 
^ static SurveyDataType cleanedUpDataType = new SurveyDataType ("cleanedUp" , true) ; 
Q static SurveyDataType proces sedDataType = new SurveyDataType ("proces sed" , true) ; 
^ static SurveyDataType publicationDataType = new 
SurveyDataType ("publication" , f alse) ; 

} 



Dziękujemy za uwagę 



